Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pubsub getting started #684

Merged
merged 25 commits into from
Aug 7, 2017
Merged

Pubsub getting started #684

merged 25 commits into from
Aug 7, 2017

Conversation

jabubake
Copy link
Contributor

Initial commit (WIP)

@jabubake jabubake added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 31, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 31, 2017
@codecov-io
Copy link

Codecov Report

Merging #684 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #684   +/-   ##
=========================================
  Coverage     58.39%   58.39%           
  Complexity      165      165           
=========================================
  Files            78       78           
  Lines          2329     2329           
  Branches        160      160           
=========================================
  Hits           1360     1360           
  Misses          922      922           
  Partials         47       47

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c0545b...011c88f. Read the comment docs.

@jabubake jabubake removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 31, 2017
@jabubake jabubake requested a review from lesv July 31, 2017 21:29
Copy link
Contributor

@lesv lesv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - we need to ask g-c-j team to actually give us typed exceptions.


// Create a new subscription
SubscriptionName subscriptionName = SubscriptionName.create(projectId, subscriptionId);
try (SubscriptionAdminClient subscriptionAdminClient = SubscriptionAdminClient.create()) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than throw an exception (above), shouldn't we catch them and explain why they might get one and what to do if they do?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


//schedule a message to be published, messages are automatically batched
private static ApiFuture<String> publishMessage(Publisher publisher, String message)
throws Exception {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than throwing an exception, shouldn't we catch them? and explain what's going on?

@lesv
Copy link
Contributor

lesv commented Aug 7, 2017

Ping

@jabubake jabubake merged commit c9a7df7 into master Aug 7, 2017
@lesv lesv deleted the pubsub_getting_started branch August 8, 2017 16:41
Sita04 pushed a commit that referenced this pull request Feb 7, 2023
…4.5 (#684)

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-pubsub](https://togithub.com/googleapis/java-pubsub) | `1.114.4` -> `1.114.5` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-pubsub/1.114.5/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-pubsub/1.114.5/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-pubsub/1.114.5/compatibility-slim/1.114.4)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-pubsub/1.114.5/confidence-slim/1.114.4)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-pubsub</summary>

### [`v1.114.5`](https://togithub.com/googleapis/java-pubsub/blob/master/CHANGELOG.md#&#8203;11145-httpswwwgithubcomgoogleapisjava-pubsubcomparev11144v11145-2021-09-27)

[Compare Source](https://togithub.com/googleapis/java-pubsub/compare/v1.114.4...v1.114.5)

</details>

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-dlp).
Sita04 pushed a commit that referenced this pull request Feb 7, 2023
🤖 I have created a release \*beep\* \*boop\*
---
### [3.0.8](https://www.github.com/googleapis/java-dlp/compare/v3.0.7...v3.0.8) (2021-09-29)


### Dependencies

* update dependency com.google.cloud:google-cloud-pubsub to v1.114.5 ([#684](https://www.github.com/googleapis/java-dlp/issues/684)) ([3a2377a](https://www.github.com/googleapis/java-dlp/commit/3a2377a4f54d5c84e023dbc08c2c40853a5f9067))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants